-
Notifications
You must be signed in to change notification settings - Fork 29
Vimmersページのリデザイン案。 #128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vimmersページのリデザイン案。 #128
Conversation
カラフルなブロックで普通に並べるような形で一から作り直した。ローカルのJekyllで確認済み。 - フォントを[Lato][1]に - ソーシャルアイコンはSVGに - jQuery使わないので削除 - 全部同じ高さなのでmasonryを削除 - DISQUSいらなそうなので削除 - Facebookのいいねライブラリも不要そうなので削除 未実装の点: - APIを使ってGitHubのアイコンを読みに行く - `short_name`の表示 - `description`の表示 - SVGのフォールバック 確認したブラウザ: - Chrome 34 - Firefox 29 - Internet Explorer 11 - Mobile Safari 7 うまく見えないであろうと思われるブラウザ: - Internet Explorer 8以下 - Android 2.3以下 [1]: https://www.google.com/fonts/specimen/Lato
|
+1 (〃l _ l)b☆ |
|
👍 |
|
きになったこと
|
|
あと、使わなくなったJSやCSSがあれば、ファイルを消しちゃって欲しいところ。 |
|
あ~ 既存の vimmers を書き換えるのではなく |
masonry を使ってた動機には高さの問題だけではなく、アニメーションもあります。 vimmers2 として新規に追加するならこれらは無くても良いです。 |
カードのサイズは小さくするのは難しいというか、ごちゃっとしてて見づらいなーというのを改善したい感じで作り始めたので、これを小さくするとコンセプトがずれてしまいます。書いてなくてすみません。 アニメーションについては、デフォルトでシャッフルしての全表示になっていること、ウィンドウ幅の切り替えはやる人がごく限られていることから削除しました。 vimmers2として同じようなページを出すのは良くないと思うので、暇な時にもうちょっと色々やってみます。 |
|
個人的にはそろそろトップもデザイン変えたい感じ On 5/11/14, MURAOKA Taro notifications@github.com wrote:
|
現状、APIで取得したデータのキャッシュはさせてないんでしたっけ? |
|
JSONPで逐一読みに行っていて、キャッシュしている様子はないですね。アクセスしたユーザーのAPI制限の消費なので、あまり良いとは思えないです。 |
|
あ、 github pages だからキャッシュさせようがないんですね… |
|
heroku 経由なので heroku で一定時間キャッシュなら可能。 On 5/11/14, Kyo Nagashima notifications@github.com wrote:
|
|
vimmers.jsonはHeroku経由ですけど、GitHubのアイコン読み込みは、動的にGitHub APIのJSONPを直接呼んでるような気がします。 |
それはそれで Issue 建てましょ…てかあった気もするけど。 |
|
現状と今後の方針のまとめ
|
これはPRに含まれてると嬉しいw |
|
なるほど、僕がコンセプトをちゃんと理解してなかったことに問題がありそうなので、この形でなら別URLに変えた方が良いですね。 元のを復活させた上で、 |
私が把握している限りではそうです。 |
このリンク「Vimmersがよりオシャレになりました!」みたいな文言とかどうだろ?w |
|
vimmersとvimmers2の相互にリンクを追加しておきます。
自分でオシャレって書くのは抵抗がありますw 追加する形のコミットログになるように、PRは作り直したほうが良さそうな気がしますが、どうでしょうか? |
面倒でなければ、お願いします!
事実だし、私は良いと思いますよw |
|
#129 でPR出しました。 |
カラフルなブロックで普通に並べるような形で一から作り直した。ローカルのJekyllで確認済み。
未実装の点:
short_nameの表示descriptionの表示確認したブラウザ:
うまく見えないであろうと思われるブラウザ:
スクリーンショット: